Skip to content
This repository was archived by the owner on Aug 14, 2019. It is now read-only.

Updated Sample Project#7

Open
kabirkhan wants to merge 3 commits intoAzure-Samples:masterfrom
kabirkhan:master
Open

Updated Sample Project#7
kabirkhan wants to merge 3 commits intoAzure-Samples:masterfrom
kabirkhan:master

Conversation

@kabirkhan
Copy link

Hi the sample was asking for the a LuisSubscriptionID. However, the API now requires a key from an azure deployment of a Cognitive Services LUIS API. I changed this wording to make it more clear to future developers.

Also, I updated the sample to use Android support 24.2.0, gradle 2.14.1 and to compile for SDK version 24. Everything else still works the same in the sample, it is now just easier to use.

@msftclas
Copy link

Hi @kabirkhan, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@msftclas
Copy link

@kabirkhan, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.

Thanks, MSBOT;

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants